Skip to content

dep: bump netty to 4.1.69.Final and tcnative to 2.0.44.Final - #8593

Closed
ran-su wants to merge 2 commits into
grpc:masterfrom
ran-su:netty.4.1.69
Closed

dep: bump netty to 4.1.69.Final and tcnative to 2.0.44.Final#8593
ran-su wants to merge 2 commits into
grpc:masterfrom
ran-su:netty.4.1.69

Conversation

@ran-su

@ran-su ran-su commented Oct 12, 2021

Copy link
Copy Markdown
Member

No description provided.

@dapengzhang0

dapengzhang0 commented Oct 14, 2021

Copy link
Copy Markdown
Contributor

I filed #8605 to track the issue of current failure. cc @sanjaypujare

@dapengzhang0 dapengzhang0 mentioned this pull request Oct 20, 2021
Closed
@dapengzhang0

dapengzhang0 commented Dec 1, 2021

Copy link
Copy Markdown
Contributor

The issue should be fixed in Netty 4.1.71.Final. Let's wait for that release.

@sergiitk

sergiitk commented Jan 7, 2022

Copy link
Copy Markdown
Member

Temporary re-opening this to build artifacts for related investigation.

@sergiitk sergiitk reopened this Jan 7, 2022
@sergiitk sergiitk added the kokoro:force-run Add this label to a PR to tell Kokoro to re-run all tests. Not generally necessary label Jan 7, 2022
@grpc-kokoro grpc-kokoro removed the kokoro:force-run Add this label to a PR to tell Kokoro to re-run all tests. Not generally necessary label Jan 7, 2022
@sergiitk

sergiitk commented Jan 7, 2022 •
edited
Loading

Copy link
Copy Markdown
Member

For some reason, build artifacts were not saved. Closing this to try from my branch #8815

@sergiitk sergiitk closed this Jan 7, 2022
@sanjaypujare

sanjaypujare commented Jan 7, 2022

Copy link
Copy Markdown
Contributor

For some reason, build artifacts were not saved. Closing this to try from my branch #8815

You mean the test reports? You may need the changes I made in https://github.com/grpc/grpc-java/pull/8813/files#diff-a515351cde89a1974efa33a43ab0012a052e0f14deff85710f296934e81cabdbR18 and https://github.com/grpc/grpc-java/pull/8813/files#diff-79ced7352cff631f4849c3a808ab5c28f76a38c8ce8bc0f8f018168e5b79d96bR11

@sergiitk

sergiitk commented Jan 7, 2022

Copy link
Copy Markdown
Member

Yes, this. I thought we already had it at least for linux_artifacts.

@sanjaypujare

sanjaypujare commented Jan 7, 2022

Copy link
Copy Markdown
Contributor

Yes, this. I thought we already had it at least for linux_artifacts.

@dapengzhang0 made this change #8801 recently for linux tests. However the artifacts are accessed differently.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ran-su @dapengzhang0 @sergiitk @sanjaypujare @grpc-kokoro